Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

docs: Use @link in @see #9348

Merged
merged 2 commits into from
Apr 8, 2023
Merged

docs: Use @link in @see #9348

merged 2 commits into from
Apr 8, 2023

Conversation

Jiralite
Copy link
Member

@Jiralite Jiralite commented Apr 8, 2023 •
edited
Loading

Please describe the changes this PR makes and why it should be merged:
As stated in @see, an explicit @link tag is required for linking. Wherever we put a link, it must be in a @link tag.

Status and versioning classification:

  • This PR only includes non-code changes, like changes to documentation, README, etc.

@vercel
Copy link

vercel bot commented Apr 8, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
discord-js-guide ⬜️ Ignored (Inspect) Apr 8, 2023 4:55pm

@vercel
Copy link

vercel bot commented Apr 8, 2023

@Jiralite is attempting to deploy a commit to the discordjs Team on Vercel.

A member of the Team first needs to authorize it.

@Jiralite Jiralite added this to the builders 1.7.0 milestone Apr 8, 2023
@codecov
Copy link

codecov bot commented Apr 8, 2023 •
edited
Loading

Codecov Report

Merging #9348 (a06552b) into main (1b9d07f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #9348   +/-   ##
=======================================
  Coverage   59.92%   59.92%           
=======================================
  Files         214      214           
  Lines       13728    13730    +2     
  Branches     1246     1246           
=======================================
+ Hits         8226     8228    +2     
  Misses       5462     5462           
  Partials       40       40
Flag Coverage Δ
brokers 66.51% <100.00%> (+0.15%) ⬆️
builders 98.25% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
packages/brokers/src/brokers/Broker.ts 96.59% <100.00%> (+0.07%) ⬆️
packages/brokers/src/brokers/redis/BaseRedis.ts 54.06% <100.00%> (ø)
...s/contextMenuCommands/ContextMenuCommandBuilder.ts 100.00% <100.00%> (ø)
.../interactions/slashCommands/SlashCommandBuilder.ts 97.72% <100.00%> (ø)
...eractions/slashCommands/SlashCommandSubcommands.ts 100.00% <100.00%> (ø)
packages/builders/src/messages/embed/Embed.ts 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@iCrawl iCrawl merged commit d66d113 into discordjs:main Apr 8, 2023
@Jiralite Jiralite deleted the docs/correct-see-tags branch April 8, 2023 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iCrawl iCrawl iCrawl approved these changes

@kyranet kyranet Awaiting requested review from kyranet

@SpaceEEC SpaceEEC Awaiting requested review from SpaceEEC

@vladfrangu vladfrangu Awaiting requested review from vladfrangu

Assignees
No one assigned
Labels
documentation packages:brokers packages:builders semver:patch
Projects
None yet
Milestone
builders 1.6.2
Development

Successfully merging this pull request may close these issues.
2 participants
@Jiralite @iCrawl